Skip to content
This repository has been archived by the owner on May 29, 2024. It is now read-only.

Remove workaround to convert ports to integers #72

Merged
merged 2 commits into from
Nov 24, 2020

Conversation

0snap
Copy link
Contributor

@0snap 0snap commented Nov 23, 2020

馃摂 Description

With the merge of tenzir/tenzir#1187 VAST now exports ports a plain integers.
This PR removes a work-around that converted the former VAST-internal representation to ints.

馃摑 Checklist

  • All user-facing changes have changelog entries.
  • The changes are reflected on docs.tenzir.com/threatbus, if necessary.
  • The PR description contains instructions for the reviewer, if necessary.

馃幆 Review Instructions

Run the modified unit-test suite.

@0snap 0snap added the enhancement An improvement of existing code label Nov 23, 2020
@0snap 0snap requested review from lava and mavam November 23, 2020 16:28
Copy link
Member

@mavam mavam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Just one left-over comment.

I didn't test this locally, but if this path is triggered by CI, I'm fine with automated testing.

apps/vast/pyvast_threatbus/test_message_mapping.py Outdated Show resolved Hide resolved
Co-authored-by: Matthias Vallentin <matthias@tenzir.com>
@0snap 0snap merged commit 311e8d0 into master Nov 24, 2020
@0snap 0snap deleted the story/ch1744/remove-port-workaround branch November 24, 2020 10:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement An improvement of existing code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants